Skip to content

London | 26-July-SDC | Ping Wang | Sprint 2 | jq#564

Open
pathywang wants to merge 5 commits into
CodeYourFuture:mainfrom
pathywang:jp
Open

London | 26-July-SDC | Ping Wang | Sprint 2 | jq#564
pathywang wants to merge 5 commits into
CodeYourFuture:mainfrom
pathywang:jp

Conversation

@pathywang

@pathywang pathywang commented Jul 10, 2026 •
edited
Loading

Copy link
Copy Markdown

Self checklist

changelist

After i completed PREP, i made my own branch from main then did all the coursework and test with Bash under file. not struggling but lost of shell tools to memorize

@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang changed the title London | 26-July-SDC|Ping Wang | Sprint 1 | JP London | 26-July-SDC|Ping Wang | Sprint 1 | jp Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang changed the title London | 26-July-SDC|Ping Wang | Sprint 1 | jp London | 26-July-SDC|Ping Wang | Sprint 2 | jp Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang changed the title London | 26-July-SDC|Ping Wang | Sprint 2 | jp London | 26-July-SDC | Ping Wang | Sprint 2 | jp Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang changed the title London | 26-July-SDC | Ping Wang | Sprint 2 | jp London | 26-July-SDC | Ping Wang | Sprint 2 | jq Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
1 similar comment
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Jul 10, 2026
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

The files changed in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the 'files changed' tab at the top of the page.

Here is an example of a file that has been changed on this branch but shouldn't be: expect/jq/script-03.sh

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@github-actions

This comment has been minimized.

Sign in to view
@pathywang pathywang added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

The files changed in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the 'files changed' tab at the top of the page.

Here is an example of a file that has been changed on this branch but shouldn't be: expect/jq/script-03.sh

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@github-actions

This comment has been minimized.

Sign in to view
@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@pathywang pathywang added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

The files changed in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the 'files changed' tab at the top of the page.

Here is an example of a file that has been changed on this branch but shouldn't be: expect/jq/script-03.sh

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jul 10, 2026
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

Results of test:
Failed script-03.sh, please check
You passed 10/11 tasks.

LonMcGregor
LonMcGregor reviewed Jul 13, 2026
View reviewed changes

@LonMcGregor LonMcGregor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The task is correct,good work

@LonMcGregor LonMcGregor added the Complete Volunteer to add when work is complete and all review comments have been addressed. label Jul 13, 2026
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

The files changed in this PR don't match what is expected for this task.

Please check that you committed the right files for the task, and that there are no accidentally committed files from other sprints.

Please review the 'files changed' tab at the top of the page.

Here is an example of a file that has been changed on this branch but shouldn't be: expect/jq/script-03.sh

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LonMcGregor LonMcGregor LonMcGregor left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pathywang @LonMcGregor